Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add framework for new tendency, revised PV diagnostics packages #1210

Closed
wants to merge 2 commits into from

Conversation

theweathermanda
Copy link
Contributor

This PR includes commits that provide the framework for a new tendencies diagnostics package and a significantly revised PV diagnostics package for MPAS-Atmosphere.

This framework includes a new Registry_tendencies.xml file and the addition of a number of new variables in Registry_pv.xml. These diagnostic variables are now connected to variable packages that are defined in Registry.xml and tied to namelist config_* options (defined in Registry_diagnostics.xml), enabling their activation at runtime. These new features and changes are described further in each commit.

This PR corresponds to PR1 described in issue #1201.

- Add tendency and PV packages declarations to Registry.xml
- Add Registry_tendencies.xml file with tendency package variables
- Update Registry_diagnostics.xml with Registry_tendencies.xml and namelist config option declarations
- Update Registry_pv.xml with all variables required to make modifications to PV diagnostics package
… variable packages

- Added mpas_atm_diagnostics_packages.F in /src/core_atmosphere/diagnostics to setup the tendency and PV diagnostics packages based on config options and ensure comptability between the config options selected in the namelist
- Modified Makefile in /src/core_atmosphere/diagnostics to reflect this new script
- Modified mpas_atm_core_interface.F to call mpas_atm_diagnostics_packages.F to setup these packages
@theweathermanda theweathermanda deleted the diag_framework branch July 9, 2024 03:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant